[minor] don't enable appcontainer loopback on Windows 11 #875
Conversation
|
There was a previous discussion about this: #840. While Win11 changed the name of the containers, I believe it also depends on which webview (IE, v1, or v2) is being used at the time of sideloading that makes is required. On top of that, enabling loopback is an administrator operation (we call a shell command that requires admin privileges). The existing pattern was to ask because it is an admin operation (and potential security risk) and once it was enabled, we didn't have to ask anymore. By not asking, we are essentially making security decision without the user's consent. On top of that, if they are not running as an administrator when we attempt to call the command, it will still fail. If they then later on sideload using a webview that requires it in order to work (I don't recall which ones do, but I'm confident webview2 doesn't), then the add-in won't load and there will be no indication as to why. Updating the container names checked and enabled would be another way to help the situation, but I think that might still be incomplete. I think it would be nice to validate which webviews (and possibly OS combination) require the loopback enabling, adjust the check to look for that specifically, point out that it requires admin privileges, and verify the behavior with different versions of the host applications (including those that we support that still only use IE webview). But so far no one has committed to take the time to do that. |
I have been running into a prompt to enable loopback every time I start debugging on Windows 11. It seems like the appcontainer names for Edge changed, but there is also no need with Windows 11 to enable loopback, as it works when I continue.
This changes the debugging start command to skip enabling loopback when running on Windows 11. It also makes a change that it should enable loopback automatically instead of prompting.
I believe either of these should resolve the issue. I'm glad to discuss what change is actually desired.
Do these changes impact command syntax of any of the packages? (e.g., add/remove command, add/remove a command parameter, or update required parameters)
NO
Do these changes impact documentation? (e.g., a tutorial on https://learn.microsoft.com/office/dev/add-ins/overview/office-add-ins)
Is loopback documented? It might require a documentation update but not sure.
Validation/testing performed:
Describe manual testing done: